rewrite GUI in customtkinter with live preview + threaded processing#3
Open
K144U wants to merge 3 commits into
Open
rewrite GUI in customtkinter with live preview + threaded processing#3K144U wants to merge 3 commits into
K144U wants to merge 3 commits into
Conversation
K144U
commented
Apr 19, 2026
K144U
commented
- three-pane layout: controls, matplotlib preview, progress/log panel
- refactor loadfiles4CIE into pure functions (no Qt, no sys.exit); callers drive progress + error reporting via callbacks
- fix white_point.csv + illuminants.csv to resolve relative to their module so the app works regardless of cwd (and when frozen)
- add ColorLab.spec so pyinstaller can bundle everything into a single double-clickable .exe
- drop PyQt5 dependency
- three-pane layout: controls, matplotlib preview, progress/log panel - refactor loadfiles4CIE into pure functions (no Qt, no sys.exit); callers drive progress + error reporting via callbacks - fix white_point.csv + illuminants.csv to resolve relative to their module so the app works regardless of cwd (and when frozen) - add ColorLab.spec so pyinstaller can bundle everything into a single double-clickable .exe - drop PyQt5 dependency
K144U
commented
Apr 19, 2026
Author
K144U
left a comment
K144U
left a comment
There was a problem hiding this comment.
hi , i am just trying to make it less confusing
- controls panel now uses askopenfilenames instead of askdirectory, so users can multi-select .csv/.xls/.xlsx/.txt files from anywhere
- ProcessingParams carries a list of filepaths; process_batch iterates it directly
- added normalize_columns so common header variants ("wavelength (nm)", "abs", "%t", etc.) are accepted
- added .xlsx support (pd.read_excel engine); openpyxl pinned in requirements
- loadfiles4CIE: fan out files with one Wavelength + multiple numeric columns into per-column single-spectrum frames before they hit CIElab. Existing single-spectrum files are unaffected. - CIE_XYZ: when Transmission max > 1.5, divide by 100 so 0-100 %T inputs produce the same fractional values that absorbance and AIPS branches do. - gitignore: ignore *.xls/*.xlsx so user spectra inputs don't get tracked.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.